Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade to Treelite 1.0.0rc1 #201

Merged
merged 3 commits into from
Jan 5, 2021

Conversation

hcho3
Copy link
Contributor

@hcho3 hcho3 commented Dec 30, 2020

Required by rapidsai/cuml#3316, which needs the 1.0.0 version of the Treelite package.

For now the RC1 version is used; once the testing is complete with rapidsai/cuml#3316, I will release the 1.0.0 version.

@hcho3 hcho3 requested a review from a team as a code owner December 30, 2020 07:31
@hcho3 hcho3 mentioned this pull request Dec 30, 2020
8 tasks
ajschmidt8
ajschmidt8 previously approved these changes Dec 30, 2020
@ajschmidt8
Copy link
Member

ajschmidt8 commented Dec 30, 2020

image

According to the conda conflicts in CI, I think the updated libgcc-ng version that's used in the new treelite package (shown above) may be conflicting with the libgcc-ng version that we use. I'm not sure the best way to resolve this. I'll have to defer to @mike-wendt or @raydouglass when they're back from holiday next week since they're more familiar with our stack.

@ajschmidt8 ajschmidt8 dismissed their stale review December 30, 2020 16:57

CI failures.

@hcho3
Copy link
Contributor Author

hcho3 commented Dec 30, 2020

@ajschmidt8 Conda-forge upgraded their compiler stack to GCC 9 back in October: https://conda-forge.org/docs/user/announcements.html

cc @jakirkham

@jakirkham
Copy link
Member

I think PR ( conda-forge/treelite-feedstock#20 ) should let us use the older compilers for now. Though we might want to start planning to move to GCC 9 where possible in RAPIDS in the future.

This reverts commit bdd9337.
@ajschmidt8 ajschmidt8 merged commit 6805cb3 into rapidsai:branch-0.18 Jan 5, 2021
@hcho3 hcho3 deleted the upgrade_treelite branch January 5, 2021 15:14
rapids-bot bot pushed a commit to rapidsai/cuml that referenced this pull request Jan 7, 2021
Depends on dmlc/treelite#235, dmlc/treelite#236, dmlc/treelite#237, dmlc/treelite#240, dmlc/treelite#241, dmlc/treelite#242, dmlc/treelite#243

Depends on rapidsai/integration#201

Closes #2160
Closes #2795
Closes #2817

TODOs
- [x] Get code review for dmlc/treelite#235 and dmlc/treelite#236 and merge them
- [x]  Get code review for dmlc/treelite#237 and merge them
- [x] Merge dmlc/treelite#240, dmlc/treelite#241, dmlc/treelite#242, dmlc/treelite#243
- [x] Run tests locally
- [x] Tag a new version of Treelite
- [x] Release a new Conda package with the new version tag
- [x] Update `meta.yml` and the integration package to use the new Treelite
- [x] Run tests (this time build with new Conda package)

Authors:
  - Hyunsu Cho <chohyu01@cs.washington.edu>

Approvers:
  - AJ Schmidt (@ajschmidt8)
  - William Hicks (@wphicks)
  - John Zedlewski (@JohnZed)

URL: #3316
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants